Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

commit-test #1822

Merged
Merged

Conversation

gpmayorga
Copy link
Contributor

Description

[description]

Fixes #(issue)

Changes and Descriptions

[List your changes here]

Checklist:

  • I have added Rust doc comments to structs, enums, traits and functions
  • I have made corresponding changes to the documentation
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works

@wischli wischli self-assigned this Apr 29, 2024
@wischli wischli added the I12-ci Issue related to CI label Apr 29, 2024
@gpmayorga gpmayorga requested a review from wischli as a code owner April 29, 2024 14:26
@gpmayorga gpmayorga requested a review from cdamian as a code owner April 29, 2024 16:40
Copy link

codecov bot commented Apr 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.62%. Comparing base (c63c370) to head (bf6e0b2).
Report is 105 commits behind head on feat/lp-integration-tests.

Additional details and impacted files
@@                      Coverage Diff                      @@
##           feat/lp-integration-tests    #1822      +/-   ##
=============================================================
+ Coverage                      48.45%   48.62%   +0.17%     
=============================================================
  Files                            168      167       -1     
  Lines                          13331    13326       -5     
=============================================================
+ Hits                            6459     6480      +21     
+ Misses                          6872     6846      -26     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@wischli wischli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI integration tests work now! The main issue was the missing forge

@wischli wischli merged commit 6e07c6c into feat/lp-integration-tests May 3, 2024
7 of 12 checks passed
@wischli wischli deleted the lp-integration-tests-gha-submodules branch May 3, 2024 06:36
@wischli wischli mentioned this pull request Jun 6, 2024
4 tasks
mustermeiszer added a commit that referenced this pull request Jun 13, 2024
* feat: wip build script

* wip: intermediate

* feat: update lp submodule

* feat: update lp submodule

* feat: intermedaite

* feat: deploy first code

* feat: add raw

* feat: add raw

* feat: first run for deploying on sol side

* feat: add notes for tomorrow

* feat: evm side setup

* update remote lp

* feat: try using local script

* feat: update lp deps

* fix: latest sol version does not have reference anymore.

* feat: erc20 deploys

* feat: set router

* feat: register assets

* feat: add currencies to local lp deployment

* feat: test AddCurrency msg

* chore: reorg

* feat: addPool test

* chore: refactor prep new tests

* chore: refactor tests

* wip: test add_tranche

* fix: apply merge

* docs: improve + show todos

* chore: finish setup tests

* fix: use correct evm currency addresses

* fix: setup requirements

* tests: finish allow, disallow investment currency

* feat: new evm exposure

* fix: add_currency and add_pool

* fix: rm unssued

* chore: clean up

* feat: allow different verifier after process outbound

* wip: adding redo failing test

* feat: failure testable

* feat: register evm contract

* fix: lp deployment

* chore: update submodule branch

* refactor: sync evm setup order with solidity

* deps: update submodule

* wip: update submodule

* wip: attempt different sourceAddress

* Fix/lp integration tests member (#1779)

* wip

* feat: update member

* feat: mint assets to investors at setup

* tests: finish add_tranche

* chore: add names for lp tranche id stable coins

* tests: finish allow_investment_currency

* tests: finish disallow_investment_currency

* fix: add evm approve to investor setup

* wip: increase invest order

* tests: finish update_tranche_token_metadata

* tests: finish update_tranche_token_price

* tests: add transfer_tranche_tokens

* tests: add transfer_tokens

* chore: apply suggestions from code review

* chore: remove increase invest test

* fmt: taplo

* wip: fix missing EVM build in target dir

* wip attempt 2: add submodule update

* wip 3: use du instead of ll debug

* wip 4: graceful panic handler ci

* wip 4

* wip 5

* wip 5

* wip 6

* wip 7

* wip

* wip

* wip

* wip 10

* wip

* wip

* wip

* wip

* wip

* wip

* wip

* wip

* wip: disable scache

* wip: use SCCACHE_RECACHE

* wip: reenable sscache

* wip: investment_collect

* fix: issue with precompile account codes

* fix: runtime generic precompile

* wip: adding 3 pool with foreign usdc

* wip: invest working, verifying collect

* feat: invest and collect

* feat: cancel investment

* wip

* feat: additional test

* Add checks for submodules in GHA

* commit-test (#1822)

* commit-test

* try with the home directory full

* testing

* supress cargo output

* keep checking submodules

* typo

* typo 2

* remove echo

* typo "s"

* Add ./ to folders in ci run

* better checks

* better runtime folder logic

* unbound variable

* slow down logs

* remove concurrency, add logs to output

* completely removed concurrency

* infinite loop

* output the contents of target dir

* log current dir and files

* break the loop when cargo finishes

* max depth of target output

* check for release folder

* remove the target output verbose

* output cargo logs into a file

* log current dir and files also in error msg

* remove the idea  of output file

* try to get rust logs working

* fix: error logs

* disable verbose cargo test logs

* fix: typo

* add rust cwd

* fix cwd for submodules dir

* debug submodules dir

* distinguish between error and not error

* more lightweight error

* debug cwd in evm.rs

* add tmp build step

* fix verbose logging

* add foundry

* add more printlns

* reduce noise strict

* revert debug changes

* docs: improve LP_SOL_SOURCES error msg and rustdocs

* fmt: taplo

---------

Co-authored-by: William Freudenberger <[email protected]>

* fix: clippy 1

* fix: clippy 2

* fix: clippy 3

* ci: re-enable submodule pulling (#1863)

* wip: try roundtrip transfer

* feat: roundtrip tests

* fix: add_tranche

* feat: tt transfer to Centrifuge

* chore: remove debug tests

* chore: rm test that is now covered in lp/transfers

* feat: move existing test to generic

* fix: eth tx after merge main

* fix: failing precompile test

* feat: enable precompile for all runtimes

* fix: transfers

* fix: after merge main

* fix: fmt with nightly

* fix: import

* fix: unneeded import

* fix: env and block

* fix: ethereum transact inte test

* chore: review suggestion use parachain_state_mut

* fix: part 1 build script

* chore: mv naming for Luis

* chore: make precompile associated type

* fix: wrong imports of Precompiles from runtimes

* chore: remove unneeded conversion H160

---------

Co-authored-by: William Freudenberger <[email protected]>
Co-authored-by: Guillermo Perez <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I12-ci Issue related to CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants